Skip to content

Removed unnecessary comma #4252

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from
Closed

Removed unnecessary comma #4252

wants to merge 1 commit into from

Conversation

allejo
Copy link
Contributor

@allejo allejo commented Sep 19, 2014

Q A
Doc fix? no
New docs? no
Applies to all
Fixed tickets

My first pull request, so I'm not entirely sure if I did this right. Anyhow, as the title suggests, the comma doesn't belong in this sentence.

@xabbuh
Copy link
Member

xabbuh commented Sep 19, 2014

Thank you @allejo. This comma really doesn't make sense.

@javiereguiluz
Copy link
Member

@allejo since this is your first pull request to this repo, welcome to the (big) family of Symfony documentation authors!

Thanks for helping us improve Symfony and we hope to see your next pull requests anytime soon :)

@weaverryan
Copy link
Member

Easy merge - thanks Vladimir! And welcome to indeed as a docs contributor :).

@allejo allejo deleted the patch-1 branch September 19, 2014 19:04
@allejo
Copy link
Contributor Author

allejo commented Sep 19, 2014

Thanks for the warm welcome, everybody!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants